Skip to content

common, server : use the same User-Agent by default#18957

Merged
angt merged 1 commit intoggml-org:masterfrom
angt:common-server-use-the-same-user-agent-by-default
Jan 20, 2026
Merged

common, server : use the same User-Agent by default#18957
angt merged 1 commit intoggml-org:masterfrom
angt:common-server-use-the-same-user-agent-by-default

Conversation

@angt
Copy link
Copy Markdown
Member

@angt angt commented Jan 20, 2026

This commit also ensures that if a custom User-Agent is used, it will be the only one sent.

This commit also ensures that if a custom User-Agent is used, it will be
the only one sent.

Signed-off-by: Adrien Gallouët <angt@huggingface.co>
Copy link
Copy Markdown
Contributor

@ngxson ngxson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the HF backend is hard-coded to only return certain info when User-Agent == "llama-cpp", so this PR will break that.

I think we should simply remove the build_info from UA, don't think it's useful for now

@angt
Copy link
Copy Markdown
Member Author

angt commented Jan 20, 2026

the HF backend is hard-coded to only return certain info when User-Agent == "llama-cpp", so this PR will break that.

I think we should simply remove the build_info from UA, don't think it's useful for now

But I've tested before submitting the PR 🤔

@angt
Copy link
Copy Markdown
Member Author

angt commented Jan 20, 2026

But I was worried about the use of llama.cpp in llama-server, hope it doesn't change the behavior in that case

@angt angt merged commit 1c7cf94 into ggml-org:master Jan 20, 2026
78 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants